Skip to content

Add missing pytest-asyncio plugin#461

Merged
TomNicholas merged 1 commit intozarr-developers:mainfrom
chuckwondo:add-pytest-asyncio
Mar 3, 2025
Merged

Add missing pytest-asyncio plugin#461
TomNicholas merged 1 commit intozarr-developers:mainfrom
chuckwondo:add-pytest-asyncio

Conversation

@chuckwondo
Copy link
Collaborator

Prevent pytest skipping async tests and eliminate the following warnings issued during unit test execution:

  • /root/micromamba/envs/virtualizarr-tests/lib/python3.11/site-packages/_pytest/config/init.py:1441: PytestConfigWarning: Unknown config option: asyncio_default_fixture_loop_scope
  • .../VirtualiZarr/virtualizarr/tests/test_writers/test_icechunk.py:642: PytestUnknownMarkWarning: Unknown pytest.mark.asyncio - is this a typo? You can register custom marks to avoid this warning - for details, see https://docs.pytest.org/en/stable/how-to/mark.html
  • /root/micromamba/envs/virtualizarr-tests/lib/python3.11/site-packages/_pytest/python.py:148: PytestUnhandledCoroutineWarning: async def functions are not natively supported and have been skipped.

Prevent pytest skipping async tests and issuing
PytestUnhandledCoroutineWarning. Also prevent PytestConfigWarning due to
asyncio_default_fixture_loop_scope config option in pyproject.toml.
Also eliminate "PytestUnknownMarkWarning: Unknown pytest.mark.asyncio"
@TomNicholas
Copy link
Member

We were skipping tests?!

@TomNicholas
Copy link
Member

You have for this to be merged as-is @chuckwondo ?

@TomNicholas TomNicholas merged commit f6edfb1 into zarr-developers:main Mar 3, 2025
9 checks passed
@chuckwondo chuckwondo deleted the add-pytest-asyncio branch March 3, 2025 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants